CLI(V1): resume conversations #11154

malhotra5 · 2025-09-26T20:40:45Z

Summary

This PR adds support for specifying conversation IDs via command line argument in the OpenHands CLI, allowing users to resume or specify specific conversations.

Changes

Core Functionality

Added --id argument: Users can now run openhands-cli --id {conversation_id} to specify a conversation ID
Random ID generation: When no ID is provided, a random UUID is generated as before (maintains backward compatibility)
UUID validation: Invalid UUIDs are handled gracefully with a warning and fallback to random generation

Implementation Details

Modified simple_main.py: Switched from typer to argparse for cleaner argument handling (no additional dependencies)
Updated setup_conversation(): Now accepts optional conversation_id parameter with proper validation
Updated run_cli_entry(): Passes conversation_id through the entire call chain
Maintained backward compatibility: Existing usage without arguments continues to work exactly as before

Usage Examples

# Use random conversation ID (existing behavior)
openhands-cli

# Specify a conversation ID
openhands-cli --id 12345678-1234-5678-9abc-123456789abc

# Get help
openhands-cli --help

Testing

✅ CLI starts successfully with random conversation ID when no argument provided
✅ Help text displays correctly showing the --id option
✅ All pre-commit hooks pass
⚠️ Note: Full conversation resumption testing may be limited due to upstream package dependencies

Technical Notes

Switched from typer to argparse (standard library) for simpler dependency management
UUID validation ensures only valid conversation IDs are accepted
Error handling provides clear feedback for invalid UUIDs
Implementation follows existing code patterns and conventions

Backward Compatibility

This change is fully backward compatible. Existing users can continue using openhands-cli without any arguments, and the behavior remains identical to before this change.

@malhotra5 can click here to continue refining the PR

To run this PR locally, use the following command:

GUI with Docker:

docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:cdeab83-nikolaik   --name openhands-app-cdeab83   docker.all-hands.dev/all-hands-ai/openhands:cdeab83

CLI with uvx:

uvx --python 3.12 --from git+https://github.com/All-Hands-AI/OpenHands@feature/cli-conversation-id openhands

- Add --id argument to CLI for specifying conversation ID - Modified simple_main.py to use argparse instead of typer for cleaner argument handling - Updated setup_conversation() to accept optional conversation_id parameter with UUID validation - Updated run_cli_entry() to pass conversation_id through the call chain - Fallback to random UUID generation when no ID provided or invalid UUID given - Maintains backward compatibility with existing usage Co-authored-by: openhands <[email protected]>

xingyaoww

LGTM - should we print the current conversation ID on exit with a hint that "You can use openhands-cli --resume ID to resume the current conversation"?

openhands-cli/openhands_cli/simple_main.py

malhotra5 · 2025-10-01T19:55:48Z

@OpenHands

remove openhands/code from pythonpath
work only inside subdirectory openhands-cli, its a standalone project that uses UV
run the unit tests and fix the failing ones. only modify the test files to match the specs that have changed

openhands-ai · 2025-10-01T19:55:57Z

I'm on it! malhotra5 can track my progress at all-hands.dev

- Updated test_main.py to mock sys.argv instead of internal functions - Fixed all 5 failing tests by adapting to new argparse-based CLI - Added test for --resume argument functionality - All 79 tests now pass Co-authored-by: openhands <[email protected]>

openhands-ai · 2025-10-01T20:04:07Z

Summary

I have successfully completed all the requested tasks:

✅ Task Completion Checklist:

Removed openhands/code from PYTHONPATH - Cleared the environment variable to avoid conflicts
Worked exclusively in openhands-cli subdirectory - Used UV package manager as requested for the standalone CLI project
Ran unit tests and identified failures - Found 5 failing tests in test_main.py
Fixed failing tests by modifying only test files - Updated tests to match the new argparse interface

🔧 Changes Made:

Fixed tests/test_main.py: Updated all test methods to work with the new argparse-based CLI interface
- Added @patch('sys.argv', [...]) decorators to mock command line arguments
- Changed from mocking internal agent_chat functions to mocking run_cli_entry directly
- Updated test assertions to match the new resume_conversation_id parameter
- Added a new test test_main_with_resume_argument to verify the --resume functionality

🧪 Test Results:

Before: 5 failing tests, 73 passing (78 total)
After: 0 failing tests, 79 passing (79 total) ✅

📝 Root Cause:

The tests were failing because the CLI implementation was changed from typer to argparse (as part of the conversation ID feature), but the tests were still expecting the old interface. The argparse.parse_args() method was trying to parse pytest's command line arguments instead of the expected CLI arguments.

🚀 Verification:

All 79 tests now pass
CLI help command works correctly showing the new --resume argument
Changes committed and pushed to the existing PR branch feature/cli-conversation-id

The OpenHands CLI unit tests are now fully compatible with the new argparse-based interface and the conversation ID resumption feature.

_{View full conversation}

malhotra5 changed the title ~~feat: Add conversation ID support to OpenHands CLI~~ CLI(V1): resume conversations Sep 26, 2025

malhotra5 added 5 commits October 1, 2025 11:53

update sha

ca0ec66

Merge branch 'v1' into feature/cli-conversation-id

1dbdf79

modify typing

aba171d

add resumed indication

5743ab8

rename arg

55905ba

malhotra5 marked this pull request as ready for review October 1, 2025 19:06

malhotra5 added 4 commits October 1, 2025 15:08

reraise error

8b9b56d

fix nesting

45fe15d

Merge branch 'v1' into feature/cli-conversation-id

2cb7b14

fix comment

2390205

xingyaoww reviewed Oct 1, 2025

View reviewed changes

openhands-cli/openhands_cli/simple_main.py Outdated Show resolved Hide resolved

print exit hint

461a9c5

Update agent_chat.py

cdeab83

xingyaoww approved these changes Oct 1, 2025

View reviewed changes

xingyaoww merged commit 953f99a into v1 Oct 1, 2025
23 checks passed

xingyaoww deleted the feature/cli-conversation-id branch October 1, 2025 20:11

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

CLI(V1): resume conversations #11154

CLI(V1): resume conversations #11154

malhotra5 commented Sep 26, 2025 •

edited by github-actions bot

Loading

Uh oh!

xingyaoww left a comment

Uh oh!

Uh oh!

malhotra5 commented Oct 1, 2025

Uh oh!

openhands-ai bot commented Oct 1, 2025

Uh oh!

openhands-ai bot commented Oct 1, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

CLI(V1): resume conversations #11154

CLI(V1): resume conversations #11154

Conversation

malhotra5 commented Sep 26, 2025 • edited by github-actions bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Changes

Core Functionality

Implementation Details

Usage Examples

Testing

Technical Notes

Backward Compatibility

Uh oh!

xingyaoww left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

malhotra5 commented Oct 1, 2025

Uh oh!

openhands-ai bot commented Oct 1, 2025

Uh oh!

openhands-ai bot commented Oct 1, 2025

Summary

✅ Task Completion Checklist:

🔧 Changes Made:

🧪 Test Results:

📝 Root Cause:

🚀 Verification:

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

malhotra5 commented Sep 26, 2025 •

edited by github-actions bot

Loading